-
Notifications
You must be signed in to change notification settings - Fork 29.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding gulp methods for targeting arm systems #10923
Conversation
Hi @aptgetmoo, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
@aptgetmoo you've tested this and it works as expected? |
Hello @joaomoreno, Best regards, |
@mong87suk You can run VSCode on a RPi with:
Making sure that you have node v6 and npm v3.8.9 installed, or the build will fail. @Tyriar I appreciate that you don't want to actively support ARM, however since the gulpfiles are already pretty undocumented, I don't think having these methods would really imply that you have support for the architecture (and, for what it's worth, there are already references in other gulpfiles to the architecture). Sorry for the late reply, and I do understand if you want to close this due to the onslaught of people with support requests for ARM if you pull this - however I think that it would certainly be a step in the right direction, and I hope you'll consider this in the future if now's not the time. |
@aptgetmoo I'm all for merging this in due to the community convenience factor. I'm curious though, can you run the package task on a development machine and then move the package to your RPi3? Our rpm and 32-bit builds are actually all done on a 64-bit Ubuntu machine. |
@Tyriar I also built my .deb on a amd64 Ubuntu machine – it does work, but as mentioned is quite buggy. I think after more people start to build for arm, these bugs will get ironed out and arm binaries could be provided from your build machine which would be an ideal scenario. As it stands though, it's much more stable just running |
Hello @aptgetmoo Best regards, I've don fast-builds as follow configuration. fast-builds error VS code error pi@KERNEL_BZ:~/Development/vscode/scripts $ ./code.sh |
@mong87suk heya, could you read the guide for how to build VSCode - I don't mean to sound confrontational but it says that you need to run an older version of npm than 3.9.0 -- but you are running 3.10. You can downgrade your npm to the correct version with |
Hello @aptgetmoo Best regards,
make: Entering directory '/home/pi/Development/vscode/node_modules/windows-mutex/build' compilation terminated. |
@mong87suk |
FWIW, I used @aptgetmoo's vscode branch to successfully run VS Code on a non-RPi ARM7 Linux machine a few days ago. Just had to install one additional missing dependency (libnss3-dev). Thanks! 🎊 |
Hello @aptgetmoo @hxlnt |
@mong87suk this is the expected behaviour. |
Hello @aptgetmoo VS code error pi@KERNEL_BZ:~/Development/vscode/scripts $ ./code.sh I've tried this way to fix the error : Linux SUID Sandbox Development I've run following line to fix the error. Put this line in may ~/.bashrc |
@mong87suk Yes optional dependencies will fail on various operating systems but because they're optional that is fine. Instead of looking for individual errors you should be looking at whether the command itself failed and exited with a non-0 exit code, you can check this by running You can safely ignore this error:
@aptgetmoo thanks for the PR/support, I'll go ahead and merge it. Please follow up if you have issues with the arm packaging 👍 |
This pull request adds gulp methods for building and packaging VSCode on ARM-based linux systems.